Skip to content
This repository has been archived by the owner on Aug 5, 2021. It is now read-only.

Adding How to Inventory Code menu option under Federal Agencies #582

Merged
merged 1 commit into from
Jun 21, 2018

Conversation

RicardoAReyes
Copy link
Contributor

@jlow81 per our conversation yesterday, I'm adding the "How to Inventory Code" menu option as the first option on the Federal Agencies submenu list. The menu option is hyperlinked to the Metadata Schema 2.0.0 requirements.

See demo:
https://files.fm/u/6des9n7g#/view/how-to-inventory-code.png

@DanielJDufour or @froi please review PR.

thanks,

Ricardo

@froi
Copy link
Contributor

froi commented Jun 21, 2018

I've pulled the branch and test locally. No errors during the tests.

The error that showed up in the CircleCI build is not related to the tests themselves. In fact, the test threw the error correctly. The issue has to do with the use of the API key being blocked/denied access.

The key is present in three files:

  • code-gov-config.json
  • code-dot-gov.json
  • your-city.json

The keys are blocked by IP/domain so they are "safe". That being said they should be removed and our tests should probably be changed to not depend on our API.

@DanielJDufour I'll be creating some issues to remove these and rethink how we are doing our tests.

@froi
Copy link
Contributor

froi commented Jun 21, 2018

@DanielJDufour
Copy link
Contributor

Awesome. Great PR! Thanks!

@DanielJDufour DanielJDufour merged commit 8e2bd88 into master Jun 21, 2018
@DanielJDufour DanielJDufour deleted the feature-menu-nav-add-inventory-link branch June 21, 2018 15:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants